Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(service-worker): various small fixes/improvements in SW docs #37555

Closed
wants to merge 3 commits into from

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Jun 12, 2020

  • Commit 1: update default value of SwRegistrationOptions#registrationStrategy
    The default value was changed from registerWhenStable to registerWhenStable:30000 in 29e8a64, but the decumentation was not updated to reflect that.

    This commit updates the documentation to mention the correct default value.

  • Commit 2: docs(service-worker): minor fixes/improvements in the SW Communication guide
    This commit includes various fixes/improvements for the "Service worker communication" guide.

This PR partially addresses #37527.

@pullapprove pullapprove bot requested a review from IgorMinar June 12, 2020 15:21
@gkalpak gkalpak added comp: docs area: service-worker Issues related to the @angular/service-worker package action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jun 12, 2020
@ngbot ngbot bot modified the milestone: needsTriage Jun 12, 2020
@mary-poppins
Copy link

You can preview 01ca0d5 at https://pr37555-01ca0d5.ngbuilds.io/.

@mary-poppins
Copy link

You can preview 9c78503 at https://pr37555-9c78503.ngbuilds.io/.

Copy link
Contributor

@kapunahelewong kapunahelewong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few style suggestions. Thank you, @gkalpak!

Reviewed-for: global-docs-approvers

aio/content/guide/service-worker-communications.md Outdated Show resolved Hide resolved
aio/content/guide/service-worker-communications.md Outdated Show resolved Hide resolved
aio/content/guide/service-worker-communications.md Outdated Show resolved Hide resolved
aio/content/guide/service-worker-communications.md Outdated Show resolved Hide resolved
@gkalpak gkalpak added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: merge The PR is ready for merge by the caretaker labels Jun 13, 2020
@gkalpak gkalpak removed the request for review from IgorMinar June 13, 2020 07:45
…registrationStrategy`

The default value was changed from `registerWhenStable` to
`registerWhenStable:30000` in 29e8a64,
but the decumentation was not updated to reflect that.

This commit updates the documentation to mention the correct default
value.
…n guide

This commit includes various fixes/improvements for the
"Service worker communication" guide.

This partially addresses angular#37527.
@gkalpak gkalpak added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Jun 13, 2020
@gkalpak gkalpak requested review from IgorMinar and removed request for IgorMinar June 13, 2020 08:01
@mary-poppins
Copy link

You can preview 96c8ec7 at https://pr37555-96c8ec7.ngbuilds.io/.

@mhevery mhevery closed this in bcc0a03 Jun 15, 2020
mhevery pushed a commit that referenced this pull request Jun 15, 2020
…n guide (#37555)

This commit includes various fixes/improvements for the
"Service worker communication" guide.

This partially addresses #37527.

PR Close #37555
mhevery pushed a commit that referenced this pull request Jun 15, 2020
…registrationStrategy` (#37555)

The default value was changed from `registerWhenStable` to
`registerWhenStable:30000` in 29e8a64,
but the decumentation was not updated to reflect that.

This commit updates the documentation to mention the correct default
value.

PR Close #37555
mhevery pushed a commit that referenced this pull request Jun 15, 2020
…n guide (#37555)

This commit includes various fixes/improvements for the
"Service worker communication" guide.

This partially addresses #37527.

PR Close #37555
@gkalpak gkalpak deleted the docs-sw-registration branch June 16, 2020 10:16
ngwattcos pushed a commit to ngwattcos/angular that referenced this pull request Jun 25, 2020
…registrationStrategy` (angular#37555)

The default value was changed from `registerWhenStable` to
`registerWhenStable:30000` in 29e8a64,
but the decumentation was not updated to reflect that.

This commit updates the documentation to mention the correct default
value.

PR Close angular#37555
ngwattcos pushed a commit to ngwattcos/angular that referenced this pull request Jun 25, 2020
…n guide (angular#37555)

This commit includes various fixes/improvements for the
"Service worker communication" guide.

This partially addresses angular#37527.

PR Close angular#37555
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jul 17, 2020
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
…registrationStrategy` (angular#37555)

The default value was changed from `registerWhenStable` to
`registerWhenStable:30000` in 29e8a64,
but the decumentation was not updated to reflect that.

This commit updates the documentation to mention the correct default
value.

PR Close angular#37555
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
…n guide (angular#37555)

This commit includes various fixes/improvements for the
"Service worker communication" guide.

This partially addresses angular#37527.

PR Close angular#37555
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: service-worker Issues related to the @angular/service-worker package cla: yes target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants